Skip to content

chore(deps): Set the correct postgres version everywhere and make failing checks neutral#178

Merged
crankynetman merged 2 commits intomainfrom
topic/chriscummings/postgres-version-this-time-with-feeling
Nov 12, 2025
Merged

chore(deps): Set the correct postgres version everywhere and make failing checks neutral#178
crankynetman merged 2 commits intomainfrom
topic/chriscummings/postgres-version-this-time-with-feeling

Conversation

@crankynetman
Copy link
Copy Markdown
Collaborator

@crankynetman crankynetman commented Nov 12, 2025

We didn't set the postgres version in the makefile and that does seem to matter. I'm not sure why we set it in two places, but it seems it was done on purpose.

Also let's make the "next python version" checks neutral checks since we know that py3.13 isn't supported until either libraries get fixed or we upgrade to django 5.2 or newer.

Note: Until this is merged into main, checks for 3.13 on push and pull_request will fail I think.

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Nov 12, 2025

File Coverage
All files 81%
config/consumers.py 78%
config/urls.py 69%
config/settings/base.py 69%
config/settings/local.py 72%
scram/route_manager/admin.py 71%
scram/route_manager/models.py 70%
scram/route_manager/views.py 86%
scram/route_manager/api/serializers.py 92%
scram/route_manager/api/views.py 74%
scram/shared/shared_code.py 56%
scram/templates/403.html 91%
scram/templates/404.html 91%
scram/templates/base.html 99%
scram/templates/route_manager/home.html 80%

Minimum allowed coverage is 50%

Generated by 🐒 cobertura-action against c924aff

…isn't supported until either libraries get fixed or we upgrade to django 5.2 or newer.
@crankynetman crankynetman changed the title chore(deps): Set the correct postgres version everywhere chore(deps): Set the correct postgres version everywhere and make failing checks neutral Nov 12, 2025
Copy link
Copy Markdown
Collaborator

@samoehlert samoehlert left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks good catch

@crankynetman crankynetman merged commit dce703c into main Nov 12, 2025
18 of 25 checks passed
@crankynetman crankynetman deleted the topic/chriscummings/postgres-version-this-time-with-feeling branch November 12, 2025 19:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants